feat(notes): blank note editor + edit flow by rtBot · Pull Request #1427 · rtCamp/next-pms

rtBot · 2026-06-03T11:57:48Z

Description

Implements the Blank Note Editor view.
Wires the "New blank note" option in the Notes tab's Create dropdown to a dedicated full-page editor (/projects/:projectId/notes/new) that creates a Note.
Adds per-card ellipsis with an "Edit" option (/projects/:projectId/notes/:noteId/edit) that loads the same editor in update mode. Save returns the user to the Notes grid (?tab=notes), where SWR revalidates on mount and shows the new / updated card.
Adds per-card ellipsis with an "Delete" option
Adds more dark border and shadows to note cards
Updates note cards to show full date formatting 1 week ago.

Screenshot/Screencast

Screen.Recording.2026-06-05.at.3.34.34.PM.mov

Checklist

I have carefully reviewed the code before submitting it for review.
This code is adequately covered by unit tests to validate its functionality.
I have conducted thorough testing to ensure it functions as intended.
A member of the QA team has reviewed and tested this PR (To be checked by QA or code reviewer)

Wires the "New blank note" dropdown option to a full-page editor view that creates Project Status Updates via the BE `create_project_status_update` endpoint, and re-instates the per-card ellipsis dropdown with an "Edit" option that opens the same editor in update mode (`update_project_status_update`). The editor hides the "About this project" sidebar and the project tabs strip for a distraction-free authoring experience, leaving only the breadcrumb + sub-header (avatar + name + "Save note") and a TextEditor workspace with placeholder text. Also fixes a pre-existing tab-order mismatch in `tabs/index.tsx` where TAB_KEYS had "notes" at index 3 but TABS had Risks at index 3 — `?tab=notes` rendered Risks content and clicks on the Notes tab generated `?tab=risks` URLs. Reordering TABS to match TAB_KEYS unbreaks the redirect target this PR depends on and the view-selector dropdown on the Risks tab. Fixes #1097 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Copilot

Pull request overview

Adds a dedicated full-page note editor under Project Details to support creating a new “blank note” and editing existing notes (Project Status Updates), including routing and a per-note card “Edit” action. Also fixes a tab ordering mismatch so ?tab=notes correctly renders the Notes tab content (needed for post-save redirect).

Changes:

Introduces /projects/:projectId/notes/new and /projects/:projectId/notes/:noteId/edit routes backed by a new note-editor/ page and supporting hooks.
Adds an ellipsis dropdown on note cards with an “Edit” action that navigates to the edit route.
Reorders the Project Details TABS array to align with TAB_KEYS so Notes/Risks tabs map correctly.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
frontend/packages/app/src/route.tsx	Registers new note editor routes and lazy-loads the editor page.
frontend/packages/app/src/pages/project-details/tabs/notes/noteCard.tsx	Turns the note card ellipsis into an actions dropdown with “Edit”.
frontend/packages/app/src/pages/project-details/tabs/index.tsx	Fixes Notes/Risks tab content ordering to match query-string tab keys.
frontend/packages/app/src/pages/project-details/note-editor/index.tsx	New full-page editor shell with edit-mode gating + save/redirect behavior.
frontend/packages/app/src/pages/project-details/note-editor/workspace.tsx	New title textarea + `TextEditor` workspace UI for the note body.
frontend/packages/app/src/pages/project-details/note-editor/subHeader.tsx	New sub-header showing avatar/name and Save button.
frontend/packages/app/src/pages/project-details/note-editor/useNoteMutation.ts	New unified create/update “save” hook for notes.
frontend/packages/app/src/pages/project-details/note-editor/useNoteDetail.ts	New note detail fetch hook for edit mode.
frontend/packages/app/src/pages/project-details/note-editor/constants.ts	New path builders for editor routes + notes grid redirect URL.
frontend/packages/app/src/pages/project-details/note-editor/types.ts	New shared types for editor mode and draft shape.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- workspace.tsx: pass fixedMenu={false} explicitly to TextEditor so the distraction-free editor never grows a toolbar regardless of upstream default changes. - useNoteMutation.ts: drop unused NoteEditorMode import flagged by lint. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

ayushnirwal · 2026-06-04T05:56:51Z

The sidebar is not visible for this page. Update about/index to look at current location for rendering

Addressed in e88cf7e — editor%2Findex.tsx (sidebar self-hides via useMatch in about/index.tsx).

Per maintainer review on PR #1427: replace the search-param view-switch in tabs/notes with real `/projects/:projectId/notes/{new,:noteId/edit}` routes nested under the project detail. - route.tsx: nest two child routes under :projectId, lazy-load NoteEditor - project-details/index.tsx: lift NotesProvider; detect editor routes via useMatch and swap the Tabs strip for <Outlet /> - about/index.tsx: split out content; return null on editor routes so the sidebar self-hides - tabs/notes/index.tsx: drop the activeView switch; Notes just renders the grid (NotesProvider lives one level up) - tabs/notes/noteCard.tsx: Dropdown Edit option navigates to the edit URL - tabs/notes/subHeader.tsx: New blank note navigates to .../notes/new - tabs/notes/editor/index.tsx: read noteId/projectId from useParams, infer mode from noteId, useNavigate to return to ?tab=notes after save - tabs/notes/constants.ts: drop NOTE_ID/NOTE_MODE search-param keys Browser-verified on PROJ-0205: - /?tab=notes renders tabs + grid + sidebar - /notes/new renders the editor full-width, no tabs/sidebar - /notes/<id>/edit pre-populates title + body, no tabs/sidebar Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

rtBot requested review from ayushnirwal and Copilot June 3, 2026 11:57

Copilot started reviewing on behalf of rtBot June 3, 2026 11:57 View session

Copilot AI reviewed Jun 3, 2026

View reviewed changes

Comment thread frontend/packages/app/src/pages/project-details/note-editor/workspace.tsx Outdated

Comment thread frontend/packages/app/src/pages/project-details/note-editor/useNoteMutation.ts Outdated

rtBot and others added 3 commits June 3, 2026 12:11

refactor: simplify notes editor

6cb5568

chore: remove unnecessary routes

ce53cc6

ayushnirwal requested changes Jun 4, 2026

View reviewed changes

b1ink0 self-requested a review June 4, 2026 13:27

b1ink0 requested changes Jun 4, 2026

View reviewed changes

Comment thread frontend/packages/app/src/route.tsx

Merge branch 'feat/redesign' into claude/feat/notes-editor

f05053e

b1ink0 reviewed Jun 4, 2026

View reviewed changes

Comment thread frontend/packages/app/src/route.tsx Outdated

b1ink0 force-pushed the claude/feat/notes-editor branch from 5611569 to f05053e Compare June 5, 2026 08:12

b1ink0 added 2 commits June 5, 2026 13:54

feat: add delete option to note

5fa8bd2

chore: change date format, add darker border and shadoow to note card

5fca995

b1ink0 requested a review from ayushnirwal June 5, 2026 10:06

ayushnirwal requested changes Jun 8, 2026

View reviewed changes

Comment thread frontend/packages/app/src/route.tsx Outdated

Comment thread frontend/packages/app/src/pages/project-details/tabs/notes/noteCard.tsx Outdated

chore: address reviews

1d10667

ayushnirwal self-requested a review June 8, 2026 07:00

ayushnirwal approved these changes Jun 8, 2026

View reviewed changes

chore: fix new blank editor loading

81603e6

b1ink0 merged commit 872ed84 into feat/redesign Jun 8, 2026
5 checks passed

b1ink0 deleted the claude/feat/notes-editor branch June 8, 2026 08:29

rtBot mentioned this pull request Jun 10, 2026

feat(notes): published note detail view #1459

Merged

4 tasks

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(notes): blank note editor + edit flow#1427

feat(notes): blank note editor + edit flow#1427
b1ink0 merged 10 commits into
feat/redesignfrom
claude/feat/notes-editor

rtBot commented Jun 3, 2026 •

edited by ayushnirwal

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

ayushnirwal Jun 4, 2026

Uh oh!

rtBot Jun 4, 2026

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

Uh oh!

Conversation

rtBot commented Jun 3, 2026 • edited by ayushnirwal Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Screenshot/Screencast

Checklist

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

ayushnirwal Jun 4, 2026

Choose a reason for hiding this comment

Uh oh!

rtBot Jun 4, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

rtBot commented Jun 3, 2026 •

edited by ayushnirwal

Loading